feat(save_and_load): add import/export methods #249

MaxNumerique · 2025-10-28T08:10:22Z

…persistence

create project manager plugin for import/export operations update tests for new functionality and async loading

…persistence create project manager plugin for import/export operations update tests for new functionality and async loading

…ns/OpenGeodeWeb-Front into feat/save_and_load

…b-Front into feat/save_and_load

plugins/project_manager.js

…ns/OpenGeodeWeb-Front into feat/save_and_load

BotellaA · 2025-10-30T18:10:15Z

composables/project_manager.js

@JulienChampagnol que penses tu de cette proposition ?

stores/app.js

stores/treeview.js

composables/project_manager.js

tests/unit/plugins/ProjectLoad.nuxt.test.js

stores/data_base.js

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/plugins/ProjectLoad.nuxt.test.js

Line 6 in 1ad0cae

default: vi.fn(() => Promise.resolve({})),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/plugins/ProjectLoad.nuxt.test.js

Line 7 in 1ad0cae

viewer_call: vi.fn(() => Promise.resolve({})),

_{reported by reviewdog 🐶}
Do not use Array#forEach

OpenGeodeWeb-Front/tests/unit/plugins/ProjectLoad.nuxt.test.js

Line 41 in 1ad0cae

.forEach((store) => stores.app.registerStore(store))

_{reported by reviewdog 🐶}
Do not use null literals

OpenGeodeWeb-Front/tests/unit/plugins/ProjectLoad.nuxt.test.js

Line 64 in 1ad0cae

selectedTree: null,

_{reported by reviewdog 🐶}
Expected { after 'if' condition.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 85 in 1ad0cae

hybridViewerStoreMock.setZScaling(snapshot.zScale)

_{reported by reviewdog 🐶}
Expected { after 'if' condition.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 116 in 1ad0cae

if (options.response_function) await options.response_function(response)

_{reported by reviewdog 🐶}
Expected !== and instead saw !=

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 84 in 1ad0cae

if (snapshot?.zScale != null)

_{reported by reviewdog 🐶}
Identifier name is too short (< 2).

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 35 in 1ad0cae

color: { r: 255, g: 255, b: 255 },

_{reported by reviewdog 🐶}
Identifier name is too short (< 2).

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 35 in 1ad0cae

color: { r: 255, g: 255, b: 255 },

_{reported by reviewdog 🐶}
Identifier name is too short (< 2).

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 35 in 1ad0cae

color: { r: 255, g: 255, b: 255 },

_{reported by reviewdog 🐶}
Identifier name is too short (< 2).

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 113 in 1ad0cae

get: (k) => (k === "new-file-name" ? "project_123.zip" : null),

_{reported by reviewdog 🐶}
Identifier name is too short (< 2).

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 161 in 1ad0cae

(v) => typeof v === "function" && v.mockClear && v.mockClear(),

_{reported by reviewdog 🐶}
The function has too many lines (85). Maximum allowed is 50.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 146 in 1ad0cae

describe("ProjectManager composable (compact)", () => {

_{reported by reviewdog 🐶}
The async function has too many lines (52). Maximum allowed is 50.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 178 in 1ad0cae

test("importProjectFile with snapshot", async () => {

_{reported by reviewdog 🐶}
Do not use null comparisons without type-checking operators.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 84 in 1ad0cae

if (snapshot?.zScale != null)

_{reported by reviewdog 🐶}
1

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 46 in 1ad0cae

focal_point: [1, 2, 3],

_{reported by reviewdog 🐶}
2

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 46 in 1ad0cae

focal_point: [1, 2, 3],

_{reported by reviewdog 🐶}
3

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 46 in 1ad0cae

focal_point: [1, 2, 3],

_{reported by reviewdog 🐶}
0

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 47 in 1ad0cae

view_up: [0, 1, 0],

_{reported by reviewdog 🐶}
1

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 47 in 1ad0cae

view_up: [0, 1, 0],

_{reported by reviewdog 🐶}
0

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 47 in 1ad0cae

view_up: [0, 1, 0],

_{reported by reviewdog 🐶}
10

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 48 in 1ad0cae

position: [10, 11, 12],

_{reported by reviewdog 🐶}
11

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 48 in 1ad0cae

position: [10, 11, 12],

_{reported by reviewdog 🐶}
12

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 48 in 1ad0cae

position: [10, 11, 12],

_{reported by reviewdog 🐶}
0.1

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 50 in 1ad0cae

clipping_range: [0.1, 1000],

_{reported by reviewdog 🐶}
1000

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 50 in 1ad0cae

clipping_range: [0.1, 1000],

_{reported by reviewdog 🐶}
4

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 190 in 1ad0cae

expect(viewer_call).toHaveBeenCalledTimes(4)

_{reported by reviewdog 🐶}
1.5

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 199 in 1ad0cae

expect(hybridViewerStoreMock.setZScaling).toHaveBeenCalledWith(1.5)

_{reported by reviewdog 🐶}
Async function has no 'await' expression.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 103 in 1ad0cae

vi.fn(async () => ({ snapshot: snapshotMock })),

_{reported by reviewdog 🐶}
Enforce test and it usage conventions

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 168 in 1ad0cae

test("exportProject", async () => {

_{reported by reviewdog 🐶}
Enforce test and it usage conventions

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 178 in 1ad0cae

test("importProjectFile with snapshot", async () => {

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 196 in 1ad0cae

expect(hybridViewerStoreMock.importStores).toHaveBeenCalledWith(

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 199 in 1ad0cae

expect(hybridViewerStoreMock.setZScaling).toHaveBeenCalledWith(1.5)

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 201 in 1ad0cae

expect(dataStyleStoreMock.importStores).toHaveBeenCalledWith(

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 204 in 1ad0cae

expect(dataStyleStoreMock.applyAllStylesFromState).toHaveBeenCalled()

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 206 in 1ad0cae

expect(dataBaseStoreMock.registerObject).toHaveBeenCalledWith("abc123")

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 207 in 1ad0cae

expect(dataBaseStoreMock.addItem).toHaveBeenCalledWith(

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 215 in 1ad0cae

expect(treeviewStoreMock.addItem).toHaveBeenCalledWith(

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 221 in 1ad0cae

expect(hybridViewerStoreMock.addItem).toHaveBeenCalledWith("abc123")

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 222 in 1ad0cae

expect(dataStyleStoreMock.addDataStyle).toHaveBeenCalledWith(

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 226 in 1ad0cae

expect(dataStyleStoreMock.applyDefaultStyle).toHaveBeenCalledWith("abc123")

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 228 in 1ad0cae

expect(hybridViewerStoreMock.remoteRender).toHaveBeenCalled()

_{reported by reviewdog 🐶}
Do not use setup or teardown hooks

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 147 in 1ad0cae

beforeEach(async () => {

_{reported by reviewdog 🐶}
Suggest using toBeCalledWith() or toHaveBeenCalledWith().

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 174 in 1ad0cae

expect(infraStoreMock.create_connection).toHaveBeenCalled()

_{reported by reviewdog 🐶}
Suggest using toBeCalledWith() or toHaveBeenCalledWith().

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 175 in 1ad0cae

expect(fileDownload).toHaveBeenCalled()

_{reported by reviewdog 🐶}
Suggest using toBeCalledWith() or toHaveBeenCalledWith().

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 188 in 1ad0cae

expect(infraStoreMock.create_connection).toHaveBeenCalled()

_{reported by reviewdog 🐶}
Suggest using toBeCalledWith() or toHaveBeenCalledWith().

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 189 in 1ad0cae

expect(viewerStoreMock.ws_connect).toHaveBeenCalled()

_{reported by reviewdog 🐶}
Suggest using toBeCalledWith() or toHaveBeenCalledWith().

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 195 in 1ad0cae

expect(hybridViewerStoreMock.initHybridViewer).toHaveBeenCalled()

_{reported by reviewdog 🐶}
Suggest using toBeCalledWith() or toHaveBeenCalledWith().

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 204 in 1ad0cae

expect(dataStyleStoreMock.applyAllStylesFromState).toHaveBeenCalled()

_{reported by reviewdog 🐶}
Suggest using toBeCalledWith() or toHaveBeenCalledWith().

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 228 in 1ad0cae

expect(hybridViewerStoreMock.remoteRender).toHaveBeenCalled()

_{reported by reviewdog 🐶}
Enforce lowercase test names

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 146 in 1ad0cae

describe("ProjectManager composable (compact)", () => {

_{reported by reviewdog 🐶}
async is not allowed

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 83 in 1ad0cae

importStores: vi.fn(async (snapshot) => {

_{reported by reviewdog 🐶}
async is not allowed

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 103 in 1ad0cae

vi.fn(async () => ({ snapshot: snapshotMock })),

_{reported by reviewdog 🐶}
async is not allowed

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 109 in 1ad0cae

api_fetch: vi.fn(async (_req, options = {}) => {

_{reported by reviewdog 🐶}
async is not allowed

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 147 in 1ad0cae

beforeEach(async () => {

_{reported by reviewdog 🐶}
async is not allowed

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 168 in 1ad0cae

test("exportProject", async () => {

_{reported by reviewdog 🐶}
async is not allowed

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 178 in 1ad0cae

test("importProjectFile with snapshot", async () => {

_{reported by reviewdog 🐶}
Optional chaining is not allowed.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 84 in 1ad0cae

if (snapshot?.zScale != null)

_{reported by reviewdog 🐶}
Optional chaining is not allowed.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 86 in 1ad0cae

if (snapshot?.camera_options) {

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 61 in 1ad0cae

const infraStoreMock = { create_connection: vi.fn(() => Promise.resolve()) }

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 62 in 1ad0cae

const viewerStoreMock = { ws_connect: vi.fn(() => Promise.resolve()) }

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 65 in 1ad0cae

importStores: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 67 in 1ad0cae

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 71 in 1ad0cae

registerObject: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 72 in 1ad0cae

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 75 in 1ad0cae

importStores: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 76 in 1ad0cae

applyAllStylesFromState: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 77 in 1ad0cae

addDataStyle: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 78 in 1ad0cae

applyDefaultStyle: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 82 in 1ad0cae

initHybridViewer: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 95 in 1ad0cae

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 106 in 1ad0cae

viewer_call: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 61 in 1ad0cae

const infraStoreMock = { create_connection: vi.fn(() => Promise.resolve()) }

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 62 in 1ad0cae

const viewerStoreMock = { ws_connect: vi.fn(() => Promise.resolve()) }

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 65 in 1ad0cae

importStores: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 67 in 1ad0cae

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 71 in 1ad0cae

registerObject: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 72 in 1ad0cae

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 75 in 1ad0cae

importStores: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 76 in 1ad0cae

applyAllStylesFromState: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 77 in 1ad0cae

addDataStyle: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 78 in 1ad0cae

applyDefaultStyle: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 82 in 1ad0cae

initHybridViewer: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 95 in 1ad0cae

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 106 in 1ad0cae

viewer_call: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Do not use Array#forEach

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 160 in 1ad0cae

Object.values(store).forEach(

_{reported by reviewdog 🐶}
Do not use null literals

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 25 in 1ad0cae

selectedTree: null,

_{reported by reviewdog 🐶}
Do not use null literals

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 36 in 1ad0cae

vertex: null,

_{reported by reviewdog 🐶}
Do not use null literals

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 84 in 1ad0cae

if (snapshot?.zScale != null)

_{reported by reviewdog 🐶}
Do not use null literals

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 113 in 1ad0cae

get: (k) => (k === "new-file-name" ? "project_123.zip" : null),

github-actions · 2025-11-11T19:03:19Z

composables/project_manager.js

+
+        treeviewStore.isImporting = true
+
+        return Promise.resolve()


_{reported by reviewdog 🐶}
Avoid nesting promises.

github-actions · 2025-11-11T19:03:19Z

composables/project_manager.js

+
+        treeviewStore.isImporting = true
+
+        return Promise.resolve()


_{reported by reviewdog 🐶}
Avoid nesting promises.

github-actions · 2025-11-11T19:03:19Z

composables/project_manager.js

+
+        treeviewStore.isImporting = true
+
+        return Promise.resolve()


_{reported by reviewdog 🐶}
Avoid nesting promises.

github-actions · 2025-11-11T19:03:19Z

composables/project_manager.js

+
+        treeviewStore.isImporting = true
+
+        return Promise.resolve()


_{reported by reviewdog 🐶}
Avoid nesting promises.

github-actions · 2025-11-11T19:03:19Z

composables/project_manager.js

+
+        treeviewStore.isImporting = true
+
+        return Promise.resolve()


_{reported by reviewdog 🐶}
Avoid nesting promises.

…ns/OpenGeodeWeb-Front into feat/save_and_load

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

_{reported by reviewdog 🐶}
0

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 47 in c58d78d

view_up: [0, 1, 0],

_{reported by reviewdog 🐶}
10

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 48 in c58d78d

position: [10, 11, 12],

_{reported by reviewdog 🐶}
11

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 48 in c58d78d

position: [10, 11, 12],

_{reported by reviewdog 🐶}
12

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 48 in c58d78d

position: [10, 11, 12],

_{reported by reviewdog 🐶}
0.1

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 50 in c58d78d

clipping_range: [0.1, 1000],

_{reported by reviewdog 🐶}
1000

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 50 in c58d78d

clipping_range: [0.1, 1000],

_{reported by reviewdog 🐶}
4

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 190 in c58d78d

expect(viewer_call).toHaveBeenCalledTimes(4)

_{reported by reviewdog 🐶}
1.5

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 199 in c58d78d

expect(hybridViewerStoreMock.setZScaling).toHaveBeenCalledWith(1.5)

_{reported by reviewdog 🐶}
Async function has no 'await' expression.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 103 in c58d78d

vi.fn(async () => ({ snapshot: snapshotMock })),

_{reported by reviewdog 🐶}
Enforce test and it usage conventions

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 168 in c58d78d

test("exportProject", async () => {

_{reported by reviewdog 🐶}
Enforce test and it usage conventions

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 178 in c58d78d

test("importProjectFile with snapshot", async () => {

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 196 in c58d78d

expect(hybridViewerStoreMock.importStores).toHaveBeenCalledWith(

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 199 in c58d78d

expect(hybridViewerStoreMock.setZScaling).toHaveBeenCalledWith(1.5)

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 201 in c58d78d

expect(dataStyleStoreMock.importStores).toHaveBeenCalledWith(

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 204 in c58d78d

expect(dataStyleStoreMock.applyAllStylesFromState).toHaveBeenCalled()

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 206 in c58d78d

expect(dataBaseStoreMock.registerObject).toHaveBeenCalledWith("abc123")

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 207 in c58d78d

expect(dataBaseStoreMock.addItem).toHaveBeenCalledWith(

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 215 in c58d78d

expect(treeviewStoreMock.addItem).toHaveBeenCalledWith(

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 221 in c58d78d

expect(hybridViewerStoreMock.addItem).toHaveBeenCalledWith("abc123")

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 222 in c58d78d

expect(dataStyleStoreMock.addDataStyle).toHaveBeenCalledWith(

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 226 in c58d78d

expect(dataStyleStoreMock.applyDefaultStyle).toHaveBeenCalledWith("abc123")

_{reported by reviewdog 🐶}
Enforces a maximum number assertion calls in a test body.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 228 in c58d78d

expect(hybridViewerStoreMock.remoteRender).toHaveBeenCalled()

_{reported by reviewdog 🐶}
Do not use setup or teardown hooks

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 147 in c58d78d

beforeEach(async () => {

_{reported by reviewdog 🐶}
Suggest using toBeCalledWith() or toHaveBeenCalledWith().

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 174 in c58d78d

expect(infraStoreMock.create_connection).toHaveBeenCalled()

_{reported by reviewdog 🐶}
Suggest using toBeCalledWith() or toHaveBeenCalledWith().

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 175 in c58d78d

expect(fileDownload).toHaveBeenCalled()

_{reported by reviewdog 🐶}
Suggest using toBeCalledWith() or toHaveBeenCalledWith().

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 188 in c58d78d

expect(infraStoreMock.create_connection).toHaveBeenCalled()

_{reported by reviewdog 🐶}
Suggest using toBeCalledWith() or toHaveBeenCalledWith().

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 189 in c58d78d

expect(viewerStoreMock.ws_connect).toHaveBeenCalled()

_{reported by reviewdog 🐶}
Suggest using toBeCalledWith() or toHaveBeenCalledWith().

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 195 in c58d78d

expect(hybridViewerStoreMock.initHybridViewer).toHaveBeenCalled()

_{reported by reviewdog 🐶}
Suggest using toBeCalledWith() or toHaveBeenCalledWith().

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 204 in c58d78d

expect(dataStyleStoreMock.applyAllStylesFromState).toHaveBeenCalled()

_{reported by reviewdog 🐶}
Suggest using toBeCalledWith() or toHaveBeenCalledWith().

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 228 in c58d78d

expect(hybridViewerStoreMock.remoteRender).toHaveBeenCalled()

_{reported by reviewdog 🐶}
Enforce lowercase test names

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 146 in c58d78d

describe("ProjectManager composable (compact)", () => {

_{reported by reviewdog 🐶}
async is not allowed

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 83 in c58d78d

importStores: vi.fn(async (snapshot) => {

_{reported by reviewdog 🐶}
async is not allowed

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 103 in c58d78d

vi.fn(async () => ({ snapshot: snapshotMock })),

_{reported by reviewdog 🐶}
async is not allowed

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 109 in c58d78d

api_fetch: vi.fn(async (_req, options = {}) => {

_{reported by reviewdog 🐶}
async is not allowed

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 147 in c58d78d

beforeEach(async () => {

_{reported by reviewdog 🐶}
async is not allowed

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 168 in c58d78d

test("exportProject", async () => {

_{reported by reviewdog 🐶}
async is not allowed

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 178 in c58d78d

test("importProjectFile with snapshot", async () => {

_{reported by reviewdog 🐶}
Optional chaining is not allowed.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 84 in c58d78d

if (snapshot?.zScale != null)

_{reported by reviewdog 🐶}
Optional chaining is not allowed.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 86 in c58d78d

if (snapshot?.camera_options) {

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 61 in c58d78d

const infraStoreMock = { create_connection: vi.fn(() => Promise.resolve()) }

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 62 in c58d78d

const viewerStoreMock = { ws_connect: vi.fn(() => Promise.resolve()) }

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 65 in c58d78d

importStores: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 67 in c58d78d

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 71 in c58d78d

registerObject: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 72 in c58d78d

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 75 in c58d78d

importStores: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 76 in c58d78d

applyAllStylesFromState: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 77 in c58d78d

addDataStyle: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 78 in c58d78d

applyDefaultStyle: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 82 in c58d78d

initHybridViewer: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 95 in c58d78d

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 106 in c58d78d

viewer_call: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 61 in c58d78d

const infraStoreMock = { create_connection: vi.fn(() => Promise.resolve()) }

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 62 in c58d78d

const viewerStoreMock = { ws_connect: vi.fn(() => Promise.resolve()) }

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 65 in c58d78d

importStores: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 67 in c58d78d

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 71 in c58d78d

registerObject: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 72 in c58d78d

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 75 in c58d78d

importStores: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 76 in c58d78d

applyAllStylesFromState: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 77 in c58d78d

addDataStyle: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 78 in c58d78d

applyDefaultStyle: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 82 in c58d78d

initHybridViewer: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 95 in c58d78d

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 106 in c58d78d

viewer_call: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Do not use Array#forEach

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 160 in c58d78d

Object.values(store).forEach(

_{reported by reviewdog 🐶}
Do not use null literals

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 25 in c58d78d

selectedTree: null,

_{reported by reviewdog 🐶}
Do not use null literals

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 36 in c58d78d

vertex: null,

_{reported by reviewdog 🐶}
Do not use null literals

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 84 in c58d78d

if (snapshot?.zScale != null)

_{reported by reviewdog 🐶}
Do not use null literals

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 113 in c58d78d

get: (k) => (k === "new-file-name" ? "project_123.zip" : null),

github-actions · 2025-11-11T19:32:45Z

tests/unit/composables/ProjectManager.nuxt.test.js

+  initHybridViewer: vi.fn(() => Promise.resolve()),
+  importStores: vi.fn(async (snapshot) => {
+    if (snapshot?.zScale != null)
+      hybridViewerStoreMock.setZScaling(snapshot.zScale)


_{reported by reviewdog 🐶}
Expected { after 'if' condition.

github-actions · 2025-11-11T19:32:46Z

tests/unit/composables/ProjectManager.nuxt.test.js

+        get: (k) => (k === "new-file-name" ? "project_123.zip" : null),
+      },
+    }
+    if (options.response_function) await options.response_function(response)


_{reported by reviewdog 🐶}
Expected { after 'if' condition.

github-actions · 2025-11-11T19:32:46Z

tests/unit/composables/ProjectManager.nuxt.test.js

+  clear: vi.fn(),
+  initHybridViewer: vi.fn(() => Promise.resolve()),
+  importStores: vi.fn(async (snapshot) => {
+    if (snapshot?.zScale != null)


_{reported by reviewdog 🐶}
Expected !== and instead saw !=

github-actions · 2025-11-11T19:32:46Z

tests/unit/composables/ProjectManager.nuxt.test.js

+          visibility: true,
+          coloring: {
+            active: "color",
+            color: { r: 255, g: 255, b: 255 },


_{reported by reviewdog 🐶}
Identifier name is too short (< 2).

github-actions · 2025-11-11T19:32:46Z

tests/unit/composables/ProjectManager.nuxt.test.js

+          visibility: true,
+          coloring: {
+            active: "color",
+            color: { r: 255, g: 255, b: 255 },


_{reported by reviewdog 🐶}
Identifier name is too short (< 2).

github-actions · 2025-11-11T19:32:46Z

tests/unit/composables/ProjectManager.nuxt.test.js

+  hybridViewer: {
+    zScale: 1.5,
+    camera_options: {
+      focal_point: [1, 2, 3],


_{reported by reviewdog 🐶}
1

github-actions · 2025-11-11T19:32:47Z

tests/unit/composables/ProjectManager.nuxt.test.js

+  hybridViewer: {
+    zScale: 1.5,
+    camera_options: {
+      focal_point: [1, 2, 3],


_{reported by reviewdog 🐶}
2

github-actions · 2025-11-11T19:32:47Z

tests/unit/composables/ProjectManager.nuxt.test.js

+  hybridViewer: {
+    zScale: 1.5,
+    camera_options: {
+      focal_point: [1, 2, 3],


_{reported by reviewdog 🐶}
3

github-actions · 2025-11-11T19:32:47Z

tests/unit/composables/ProjectManager.nuxt.test.js

+    zScale: 1.5,
+    camera_options: {
+      focal_point: [1, 2, 3],
+      view_up: [0, 1, 0],


_{reported by reviewdog 🐶}
0

github-actions · 2025-11-11T19:32:47Z

tests/unit/composables/ProjectManager.nuxt.test.js

+    zScale: 1.5,
+    camera_options: {
+      focal_point: [1, 2, 3],
+      view_up: [0, 1, 0],


_{reported by reviewdog 🐶}
1

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

_{reported by reviewdog 🐶}
Enforce lowercase test names

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 146 in f20ac1a

describe("ProjectManager composable (compact)", () => {

_{reported by reviewdog 🐶}
async is not allowed

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 83 in f20ac1a

importStores: vi.fn(async (snapshot) => {

_{reported by reviewdog 🐶}
async is not allowed

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 103 in f20ac1a

vi.fn(async () => ({ snapshot: snapshotMock })),

_{reported by reviewdog 🐶}
async is not allowed

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 109 in f20ac1a

api_fetch: vi.fn(async (_req, options = {}) => {

_{reported by reviewdog 🐶}
async is not allowed

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 147 in f20ac1a

beforeEach(async () => {

_{reported by reviewdog 🐶}
async is not allowed

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 168 in f20ac1a

test("exportProject", async () => {

_{reported by reviewdog 🐶}
async is not allowed

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 178 in f20ac1a

test("importProjectFile with snapshot", async () => {

_{reported by reviewdog 🐶}
Optional chaining is not allowed.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 84 in f20ac1a

if (snapshot?.zScale != null)

_{reported by reviewdog 🐶}
Optional chaining is not allowed.

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 86 in f20ac1a

if (snapshot?.camera_options) {

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 61 in f20ac1a

const infraStoreMock = { create_connection: vi.fn(() => Promise.resolve()) }

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 62 in f20ac1a

const viewerStoreMock = { ws_connect: vi.fn(() => Promise.resolve()) }

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 65 in f20ac1a

importStores: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 67 in f20ac1a

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 71 in f20ac1a

registerObject: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 72 in f20ac1a

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 75 in f20ac1a

importStores: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 76 in f20ac1a

applyAllStylesFromState: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 77 in f20ac1a

addDataStyle: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 78 in f20ac1a

applyDefaultStyle: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 82 in f20ac1a

initHybridViewer: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 95 in f20ac1a

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 106 in f20ac1a

viewer_call: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 61 in f20ac1a

const infraStoreMock = { create_connection: vi.fn(() => Promise.resolve()) }

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 62 in f20ac1a

const viewerStoreMock = { ws_connect: vi.fn(() => Promise.resolve()) }

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 65 in f20ac1a

importStores: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 67 in f20ac1a

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 71 in f20ac1a

registerObject: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 72 in f20ac1a

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 75 in f20ac1a

importStores: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 76 in f20ac1a

applyAllStylesFromState: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 77 in f20ac1a

addDataStyle: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 78 in f20ac1a

applyDefaultStyle: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 82 in f20ac1a

initHybridViewer: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 95 in f20ac1a

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 106 in f20ac1a

viewer_call: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Do not use Array#forEach

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 160 in f20ac1a

Object.values(store).forEach(

_{reported by reviewdog 🐶}
Do not use null literals

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 25 in f20ac1a

selectedTree: null,

_{reported by reviewdog 🐶}
Do not use null literals

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 36 in f20ac1a

vertex: null,

_{reported by reviewdog 🐶}
Do not use null literals

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 84 in f20ac1a

if (snapshot?.zScale != null)

_{reported by reviewdog 🐶}
Do not use null literals

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 113 in f20ac1a

get: (k) => (k === "new-file-name" ? "project_123.zip" : null),

github-actions · 2025-11-11T19:42:35Z

tests/unit/composables/ProjectManager.nuxt.test.js

+    zScale: 1.5,
+    camera_options: {
+      focal_point: [1, 2, 3],
+      view_up: [0, 1, 0],


_{reported by reviewdog 🐶}
0

github-actions · 2025-11-11T19:42:35Z

tests/unit/composables/ProjectManager.nuxt.test.js

+    camera_options: {
+      focal_point: [1, 2, 3],
+      view_up: [0, 1, 0],
+      position: [10, 11, 12],


_{reported by reviewdog 🐶}
10

github-actions · 2025-11-11T19:42:35Z

tests/unit/composables/ProjectManager.nuxt.test.js

+    camera_options: {
+      focal_point: [1, 2, 3],
+      view_up: [0, 1, 0],
+      position: [10, 11, 12],


_{reported by reviewdog 🐶}
11

github-actions · 2025-11-11T19:42:35Z

tests/unit/composables/ProjectManager.nuxt.test.js

+    camera_options: {
+      focal_point: [1, 2, 3],
+      view_up: [0, 1, 0],
+      position: [10, 11, 12],


_{reported by reviewdog 🐶}
12

github-actions · 2025-11-11T19:42:35Z

tests/unit/composables/ProjectManager.nuxt.test.js

+      view_up: [0, 1, 0],
+      position: [10, 11, 12],
+      view_angle: 30,
+      clipping_range: [0.1, 1000],


_{reported by reviewdog 🐶}
0.1

github-actions · 2025-11-11T19:42:37Z

tests/unit/composables/ProjectManager.nuxt.test.js

+
+    const { viewer_call } = await import("@/composables/viewer_call.js")
+
+    expect(infraStoreMock.create_connection).toHaveBeenCalled()


_{reported by reviewdog 🐶}
Suggest using toBeCalledWith() or toHaveBeenCalledWith().

github-actions · 2025-11-11T19:42:37Z

tests/unit/composables/ProjectManager.nuxt.test.js

+    const { viewer_call } = await import("@/composables/viewer_call.js")
+
+    expect(infraStoreMock.create_connection).toHaveBeenCalled()
+    expect(viewerStoreMock.ws_connect).toHaveBeenCalled()


_{reported by reviewdog 🐶}
Suggest using toBeCalledWith() or toHaveBeenCalledWith().

github-actions · 2025-11-11T19:42:38Z

tests/unit/composables/ProjectManager.nuxt.test.js

+    expect(treeviewStoreMock.importStores).toHaveBeenCalledWith(
+      snapshotMock.treeview,
+    )
+    expect(hybridViewerStoreMock.initHybridViewer).toHaveBeenCalled()


_{reported by reviewdog 🐶}
Suggest using toBeCalledWith() or toHaveBeenCalledWith().

github-actions · 2025-11-11T19:42:38Z

tests/unit/composables/ProjectManager.nuxt.test.js

+    expect(dataStyleStoreMock.importStores).toHaveBeenCalledWith(
+      snapshotMock.dataStyle,
+    )
+    expect(dataStyleStoreMock.applyAllStylesFromState).toHaveBeenCalled()


_{reported by reviewdog 🐶}
Suggest using toBeCalledWith() or toHaveBeenCalledWith().

github-actions · 2025-11-11T19:42:38Z

tests/unit/composables/ProjectManager.nuxt.test.js

+    )
+    expect(dataStyleStoreMock.applyDefaultStyle).toHaveBeenCalledWith("abc123")
+
+    expect(hybridViewerStoreMock.remoteRender).toHaveBeenCalled()


_{reported by reviewdog 🐶}
Suggest using toBeCalledWith() or toHaveBeenCalledWith().

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 78 in 16a5aa9

applyDefaultStyle: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 82 in 16a5aa9

initHybridViewer: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 95 in 16a5aa9

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Expected catch or return

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 106 in 16a5aa9

viewer_call: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 61 in 16a5aa9

const infraStoreMock = { create_connection: vi.fn(() => Promise.resolve()) }

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 62 in 16a5aa9

const viewerStoreMock = { ws_connect: vi.fn(() => Promise.resolve()) }

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 65 in 16a5aa9

importStores: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 67 in 16a5aa9

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 71 in 16a5aa9

registerObject: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 72 in 16a5aa9

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 75 in 16a5aa9

importStores: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 76 in 16a5aa9

applyAllStylesFromState: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 77 in 16a5aa9

addDataStyle: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 78 in 16a5aa9

applyDefaultStyle: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 82 in 16a5aa9

initHybridViewer: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 95 in 16a5aa9

addItem: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 106 in 16a5aa9

viewer_call: vi.fn(() => Promise.resolve()),

_{reported by reviewdog 🐶}
Do not use Array#forEach

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 160 in 16a5aa9

Object.values(store).forEach(

_{reported by reviewdog 🐶}
Do not use null literals

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 25 in 16a5aa9

selectedTree: null,

_{reported by reviewdog 🐶}
Do not use null literals

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 36 in 16a5aa9

vertex: null,

_{reported by reviewdog 🐶}
Do not use null literals

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 84 in 16a5aa9

if (snapshot?.zScale != null)

_{reported by reviewdog 🐶}
Do not use null literals

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 113 in 16a5aa9

get: (k) => (k === "new-file-name" ? "project_123.zip" : null),

github-actions · 2025-11-11T19:46:21Z

tests/unit/composables/ProjectManager.nuxt.test.js

+  exportStores: vi.fn(() => ({ projectName: "mockedProject" })),
+}))
+
+describe("ProjectManager composable (compact)", () => {


_{reported by reviewdog 🐶}
Enforce lowercase test names

github-actions · 2025-11-11T19:46:21Z

tests/unit/composables/ProjectManager.nuxt.test.js

+const hybridViewerStoreMock = {
+  clear: vi.fn(),
+  initHybridViewer: vi.fn(() => Promise.resolve()),
+  importStores: vi.fn(async (snapshot) => {


_{reported by reviewdog 🐶}
async is not allowed

github-actions · 2025-11-11T19:46:21Z

tests/unit/composables/ProjectManager.nuxt.test.js

+// Mocks
+vi.stubGlobal(
+  "$fetch",
+  vi.fn(async () => ({ snapshot: snapshotMock })),


_{reported by reviewdog 🐶}
async is not allowed

github-actions · 2025-11-11T19:46:21Z

tests/unit/composables/ProjectManager.nuxt.test.js

+  viewer_call: vi.fn(() => Promise.resolve()),
+}))
+vi.mock("@/composables/api_fetch.js", () => ({
+  api_fetch: vi.fn(async (_req, options = {}) => {


_{reported by reviewdog 🐶}
async is not allowed

github-actions · 2025-11-11T19:46:21Z

tests/unit/composables/ProjectManager.nuxt.test.js

+}))
+
+describe("ProjectManager composable (compact)", () => {
+  beforeEach(async () => {


_{reported by reviewdog 🐶}
async is not allowed

github-actions · 2025-11-11T19:46:22Z

tests/unit/composables/ProjectManager.nuxt.test.js

+}
+const dataBaseStoreMock = {
+  clear: vi.fn(),
+  registerObject: vi.fn(() => Promise.resolve()),


_{reported by reviewdog 🐶}
Expected catch or return

github-actions · 2025-11-11T19:46:22Z

tests/unit/composables/ProjectManager.nuxt.test.js

+const dataBaseStoreMock = {
+  clear: vi.fn(),
+  registerObject: vi.fn(() => Promise.resolve()),
+  addItem: vi.fn(() => Promise.resolve()),


_{reported by reviewdog 🐶}
Expected catch or return

github-actions · 2025-11-11T19:46:22Z

tests/unit/composables/ProjectManager.nuxt.test.js

+  addItem: vi.fn(() => Promise.resolve()),
+}
+const dataStyleStoreMock = {
+  importStores: vi.fn(() => Promise.resolve()),


_{reported by reviewdog 🐶}
Expected catch or return

github-actions · 2025-11-11T19:46:22Z

tests/unit/composables/ProjectManager.nuxt.test.js

+}
+const dataStyleStoreMock = {
+  importStores: vi.fn(() => Promise.resolve()),
+  applyAllStylesFromState: vi.fn(() => Promise.resolve()),


_{reported by reviewdog 🐶}
Expected catch or return

github-actions · 2025-11-11T19:46:22Z

tests/unit/composables/ProjectManager.nuxt.test.js

+const dataStyleStoreMock = {
+  importStores: vi.fn(() => Promise.resolve()),
+  applyAllStylesFromState: vi.fn(() => Promise.resolve()),
+  addDataStyle: vi.fn(() => Promise.resolve()),


_{reported by reviewdog 🐶}
Expected catch or return

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

_{reported by reviewdog 🐶}
Do not use null literals

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 25 in 10dcaad

selectedTree: null,

_{reported by reviewdog 🐶}
Do not use null literals

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 36 in 10dcaad

vertex: null,

_{reported by reviewdog 🐶}
Do not use null literals

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 84 in 10dcaad

if (snapshot?.zScale != null)

_{reported by reviewdog 🐶}
Do not use null literals

OpenGeodeWeb-Front/tests/unit/composables/ProjectManager.nuxt.test.js

Line 113 in 10dcaad

get: (k) => (k === "new-file-name" ? "project_123.zip" : null),

github-actions · 2025-11-11T19:54:37Z

tests/unit/plugins/project_load.nuxt.test.js

+      hybrid: useHybridViewerStore(),
+    }
+    Object.values(stores)
+      .slice(1)


_{reported by reviewdog 🐶}
1

github-actions · 2025-11-11T19:54:37Z

tests/unit/plugins/project_load.nuxt.test.js

+})
+
+describe("Project import", () => {
+  test("app.importStores restores stores", async () => {


_{reported by reviewdog 🐶}
Enforce test and it usage conventions

github-actions · 2025-11-11T19:54:37Z

tests/unit/plugins/project_load.nuxt.test.js

+  }),
+}))
+
+beforeEach(() => {


_{reported by reviewdog 🐶}
Do not use setup or teardown hooks

github-actions · 2025-11-11T19:54:37Z

tests/unit/plugins/project_load.nuxt.test.js

+  )
+})
+
+describe("Project import", () => {


_{reported by reviewdog 🐶}
Enforce lowercase test names

github-actions · 2025-11-11T19:54:37Z

tests/unit/plugins/project_load.nuxt.test.js

+  }),
+}))
+
+beforeEach(() => {


_{reported by reviewdog 🐶}
Require test cases and hooks to be inside a describe block

github-actions · 2025-11-11T19:54:39Z

tests/unit/composables/ProjectManager.nuxt.test.js

+  importStores: vi.fn(() => Promise.resolve()),
+  applyAllStylesFromState: vi.fn(() => Promise.resolve()),
+  addDataStyle: vi.fn(() => Promise.resolve()),
+  applyDefaultStyle: vi.fn(() => Promise.resolve()),


_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

github-actions · 2025-11-11T19:54:40Z

tests/unit/composables/ProjectManager.nuxt.test.js

+}
+const hybridViewerStoreMock = {
+  clear: vi.fn(),
+  initHybridViewer: vi.fn(() => Promise.resolve()),


_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

github-actions · 2025-11-11T19:54:40Z

tests/unit/composables/ProjectManager.nuxt.test.js

+      hybridViewerStoreMock.remoteRender()
+    }
+  }),
+  addItem: vi.fn(() => Promise.resolve()),


_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

github-actions · 2025-11-11T19:54:40Z

tests/unit/composables/ProjectManager.nuxt.test.js

+  vi.fn(async () => ({ snapshot: snapshotMock })),
+)
+vi.mock("@/composables/viewer_call.js", () => ({
+  viewer_call: vi.fn(() => Promise.resolve()),


_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

github-actions · 2025-11-11T19:54:40Z

tests/unit/composables/ProjectManager.nuxt.test.js

+      dataStyleStoreMock,
+      hybridViewerStoreMock,
+    ]) {
+      Object.values(store).forEach(


_{reported by reviewdog 🐶}
Do not use Array#forEach

github-actions · 2025-11-12T08:49:52Z

stores/treeview.js

+  function finalizeImportSelection() {
+    const ids = pendingSelectionIds.value || []
+    const rebuilt = []
+    if (!ids.length) {


_{reported by reviewdog 🐶}
Unexpected negated condition.

github-actions · 2025-11-12T08:49:52Z

stores/treeview.js

+    components_selection.value = []
+    pendingSelectionIds.value = []
+    model_id.value = ""
+    selectedTree.value = undefined


_{reported by reviewdog 🐶}
Unexpected use of undefined

github-actions · 2025-11-12T08:49:53Z

stores/treeview.js

+  function finalizeImportSelection() {
+    const ids = pendingSelectionIds.value || []
+    const rebuilt = []
+    if (!ids.length) {


_{reported by reviewdog 🐶}
Use .length === 0 when checking length is zero.

github-actions · 2025-11-12T08:49:53Z

tests/unit/composables/ProjectManager.nuxt.test.js

+    panelWidth: 320,
+    model_id: "",
+    isTreeCollection: false,
+    selectedTree: null,


_{reported by reviewdog 🐶}
Do not use null literals

github-actions · 2025-11-12T08:49:53Z

tests/unit/composables/ProjectManager.nuxt.test.js

+          coloring: {
+            active: "color",
+            color: { r: 255, g: 255, b: 255 },
+            vertex: null,


_{reported by reviewdog 🐶}
Do not use null literals

github-actions · 2025-11-12T08:49:53Z

tests/unit/composables/ProjectManager.nuxt.test.js

+  clear: vi.fn(),
+  initHybridViewer: vi.fn(() => Promise.resolve()),
+  importStores: vi.fn(async (snapshot) => {
+    if (snapshot?.zScale != null)


_{reported by reviewdog 🐶}
Do not use null literals

github-actions · 2025-11-12T08:49:53Z

tests/unit/composables/ProjectManager.nuxt.test.js

+    const response = {
+      _data: new Blob(["zipcontent"], { type: "application/zip" }),
+      headers: {
+        get: (k) => (k === "new-file-name" ? "project_123.zip" : null),


_{reported by reviewdog 🐶}
Do not use null literals

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

_{reported by reviewdog 🐶}
Named exports are not allowed.

OpenGeodeWeb-Front/composables/project_manager.js

Line 6 in edc66de

export function useProjectManager() {

_{reported by reviewdog 🐶}
Prefer named exports

OpenGeodeWeb-Front/composables/project_manager.js

Line 146 in edc66de

export default useProjectManager

_{reported by reviewdog 🐶}
Each then() should return a value or throw

OpenGeodeWeb-Front/composables/project_manager.js

Line 133 in edc66de

.then(function () {

_{reported by reviewdog 🐶}
Avoid nesting promises.

OpenGeodeWeb-Front/composables/project_manager.js

Line 87 in edc66de

return Promise.resolve()

_{reported by reviewdog 🐶}
Avoid nesting promises.

OpenGeodeWeb-Front/composables/project_manager.js

Line 87 in edc66de

return Promise.resolve()

_{reported by reviewdog 🐶}
Avoid nesting promises.

OpenGeodeWeb-Front/composables/project_manager.js

Line 87 in edc66de

return Promise.resolve()

_{reported by reviewdog 🐶}
Avoid nesting promises.

OpenGeodeWeb-Front/composables/project_manager.js

Line 87 in edc66de

return Promise.resolve()

_{reported by reviewdog 🐶}
Avoid nesting promises.

OpenGeodeWeb-Front/composables/project_manager.js

Line 87 in edc66de

return Promise.resolve()

_{reported by reviewdog 🐶}
Avoid nesting promises.

OpenGeodeWeb-Front/composables/project_manager.js

Line 87 in edc66de

return Promise.resolve()

_{reported by reviewdog 🐶}
Avoid nesting promises.

OpenGeodeWeb-Front/composables/project_manager.js

Line 87 in edc66de

return Promise.resolve()

_{reported by reviewdog 🐶}
Avoid nesting promises.

OpenGeodeWeb-Front/composables/project_manager.js

Line 87 in edc66de

return Promise.resolve()

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/composables/project_manager.js

Line 19 in edc66de

.then(function () {

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/composables/project_manager.js

Line 82 in edc66de

.then(function (result) {

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/composables/project_manager.js

Line 66 in edc66de

.then(function () {

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/composables/project_manager.js

Line 60 in edc66de

.then(function () {

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/composables/project_manager.js

Line 54 in edc66de

.then(function () {

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/composables/project_manager.js

Line 51 in edc66de

.then(function () {

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/composables/project_manager.js

Line 129 in edc66de

.then(function () {

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/composables/project_manager.js

Line 125 in edc66de

.then(function () {

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/composables/project_manager.js

Line 122 in edc66de

.then(function () {

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/composables/project_manager.js

Line 97 in edc66de

.then(function () {

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/composables/project_manager.js

Line 94 in edc66de

.then(function () {

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/composables/project_manager.js

Line 91 in edc66de

.then(function () {

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/composables/project_manager.js

Line 88 in edc66de

.then(function () {

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/composables/project_manager.js

Line 87 in edc66de

return Promise.resolve()

github-actions · 2025-11-12T10:11:00Z

composables/project_manager.js

+import viewer_schemas from "@geode/opengeodeweb-viewer/opengeodeweb_viewer_schemas.json"
+import { once } from "lodash"
+
+export function useProjectManager() {


_{reported by reviewdog 🐶}
Expected a function expression.

github-actions · 2025-11-12T10:11:01Z

composables/project_manager.js

+  const geode = useGeodeStore()
+  const appStore = useAppStore()
+
+  const exportProject = function () {


_{reported by reviewdog 🐶}
Unexpected unnamed function.

github-actions · 2025-11-12T10:11:01Z

composables/project_manager.js

+
+    return infraStore
+      .create_connection()
+      .then(function () {


_{reported by reviewdog 🐶}
Unexpected unnamed function.

github-actions · 2025-11-12T10:11:01Z

composables/project_manager.js

+        return api_fetch(
+          { schema, params: { snapshot, filename: defaultName } },
+          {
+            response_function: once(function (response) {


_{reported by reviewdog 🐶}
Unexpected unnamed function.

github-actions · 2025-11-12T10:11:01Z

composables/project_manager.js

+          },
+        )
+      })
+      .finally(function () {


_{reported by reviewdog 🐶}
Unexpected unnamed function.

github-actions · 2025-11-12T10:11:03Z

composables/project_manager.js

+                : {}
+            const items = Object.entries(snapshotDataBase).map(function (pair) {
+              const id = pair[0]
+              const item = pair[1]


_{reported by reviewdog 🐶}
1

github-actions · 2025-11-12T10:11:04Z

composables/project_manager.js

+              const binaryLightViewable =
+                item && item.vtk_js && item.vtk_js.binary_light_viewable
+                  ? item.vtk_js.binary_light_viewable
+                  : undefined


_{reported by reviewdog 🐶}
Unexpected use of undefined

github-actions · 2025-11-12T10:11:04Z

composables/project_manager.js

+                ? snapshot.dataBase.db
+                : {}
+            const items = Object.entries(snapshotDataBase).map(function (pair) {
+              const id = pair[0]


_{reported by reviewdog 🐶}
Use Array destructuring.

github-actions · 2025-11-12T10:11:04Z

composables/project_manager.js

+                : {}
+            const items = Object.entries(snapshotDataBase).map(function (pair) {
+              const id = pair[0]
+              const item = pair[1]


_{reported by reviewdog 🐶}
Use Array destructuring.

github-actions · 2025-11-12T10:11:04Z

composables/project_manager.js

+import back_schemas from "@geode/opengeodeweb-back/opengeodeweb_back_schemas.json"
+import fileDownload from "js-file-download"
+import viewer_schemas from "@geode/opengeodeweb-viewer/opengeodeweb_viewer_schemas.json"
+import { once } from "lodash"


_{reported by reviewdog 🐶}
Imports should be sorted alphabetically.

…b-Front into feat/save_and_load

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/composables/project_manager.js

Line 97 in 3d89884

.then(function () {

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/composables/project_manager.js

Line 94 in 3d89884

.then(function () {

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/composables/project_manager.js

Line 91 in 3d89884

.then(function () {

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/composables/project_manager.js

Line 88 in 3d89884

.then(function () {

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/composables/project_manager.js

Line 87 in 3d89884

return Promise.resolve()

_{reported by reviewdog 🐶}
Unexpected block statement surrounding arrow body; move the returned value immediately after the =>.

OpenGeodeWeb-Front/stores/data_style.js

Line 54 in 3d89884

const setModelEdgesVisibility = (id, visibility) => {

_{reported by reviewdog 🐶}
Unexpected block statement surrounding arrow body; move the returned value immediately after the =>.

OpenGeodeWeb-Front/stores/data_style.js

Line 58 in 3d89884

const modelEdgesVisibility = (id) => {

_{reported by reviewdog 🐶}
Unexpected block statement surrounding arrow body; move the returned value immediately after the =>.

OpenGeodeWeb-Front/stores/data_style.js

Line 62 in 3d89884

const exportStores = () => {

_{reported by reviewdog 🐶}
Optional chaining is not allowed.

OpenGeodeWeb-Front/stores/data_style.js

Line 81 in 3d89884

const objectType = meta?.object_type

_{reported by reviewdog 🐶}
Each then() should return a value or throw

OpenGeodeWeb-Front/stores/data_style.js

Line 27 in 3d89884

]).then(() => {

_{reported by reviewdog 🐶}
Each then() should return a value or throw

OpenGeodeWeb-Front/stores/data_style.js

Line 33 in 3d89884

]).then(() => {

_{reported by reviewdog 🐶}
Each then() should return a value or throw

OpenGeodeWeb-Front/stores/data_style.js

Line 43 in 3d89884

return meshStyleStore.applyMeshStyle(id).then(() => {

_{reported by reviewdog 🐶}
Each then() should return a value or throw

OpenGeodeWeb-Front/stores/data_style.js

Line 47 in 3d89884

return modelStyleStore.applyModelStyle(id).then(() => {

_{reported by reviewdog 🐶}
Each then() should return a value or throw

OpenGeodeWeb-Front/stores/data_style.js

Line 89 in 3d89884

return Promise.all(promises).then(() => {

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/stores/data_style.js

Line 25 in 3d89884

return Promise.all([

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/stores/data_style.js

Line 31 in 3d89884

return Promise.all([

_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

OpenGeodeWeb-Front/stores/data_style.js

Line 89 in 3d89884

return Promise.all(promises).then(() => {

_{reported by reviewdog 🐶}
Wrapping single-element array with Promise.all() is unnecessary.

OpenGeodeWeb-Front/stores/data_style.js

Line 25 in 3d89884

return Promise.all([

_{reported by reviewdog 🐶}
Wrapping single-element array with Promise.all() is unnecessary.

OpenGeodeWeb-Front/stores/data_style.js

Line 31 in 3d89884

return Promise.all([

github-actions · 2025-11-12T12:41:53Z

utils/default_styles.js

 const surfaces_defaultColor = { r: 20, g: 20, b: 20 }
 const blocks_defaultVisibility = true
-const blocks_defaultColor = { r: 20, g: 20, b: 20 }
+const blocks_defaultColor = { r: 255, g: 255, b: 255 }


_{reported by reviewdog 🐶}
Identifier name is too short (< 2).

github-actions · 2025-11-12T12:41:53Z

utils/default_styles.js

 const surfaces_defaultColor = { r: 20, g: 20, b: 20 }
 const blocks_defaultVisibility = true
-const blocks_defaultColor = { r: 20, g: 20, b: 20 }
+const blocks_defaultColor = { r: 255, g: 255, b: 255 }


_{reported by reviewdog 🐶}
Identifier name is too short (< 2).

github-actions · 2025-11-12T12:41:54Z

utils/default_styles.js

 const surfaces_defaultColor = { r: 20, g: 20, b: 20 }
 const blocks_defaultVisibility = true
-const blocks_defaultColor = { r: 20, g: 20, b: 20 }
+const blocks_defaultColor = { r: 255, g: 255, b: 255 }


_{reported by reviewdog 🐶}
Identifier name is too short (< 2).

github-actions · 2025-11-12T12:41:54Z

composables/project_manager.js

+import viewer_schemas from "@geode/opengeodeweb-viewer/opengeodeweb_viewer_schemas.json"
+import { once } from "lodash"
+
+export function useProjectManager() {


_{reported by reviewdog 🐶}
Named exports are not allowed.

github-actions · 2025-11-12T12:41:54Z

composables/project_manager.js

+  return { exportProject, importProjectFile }
+}
+
+export default useProjectManager


_{reported by reviewdog 🐶}
Prefer named exports

github-actions · 2025-11-12T12:41:57Z

composables/project_manager.js

+      .then(function () {
+        return viewerStore.ws_connect()
+      })
+      .then(function () {


_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

github-actions · 2025-11-12T12:41:57Z

composables/project_manager.js

+
+    return infraStore
+      .create_connection()
+      .then(function () {


_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

github-actions · 2025-11-12T12:41:58Z

composables/project_manager.js

+            const dataStyleStore = useDataStyleStore()
+            return dataStyleStore.importStores(snapshot.dataStyle)
+          })
+          .then(function () {


_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

github-actions · 2025-11-12T12:41:58Z

composables/project_manager.js

+          .then(function () {
+            return hybridViewerStore.importStores(snapshot.hybridViewer)
+          })
+          .then(function () {


_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

github-actions · 2025-11-12T12:41:58Z

composables/project_manager.js

+
+            return importWorkflowFromSnapshot(items)
+          })
+          .then(function () {


_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

…ns/OpenGeodeWeb-Front into feat/save_and_load

github-actions · 2025-11-12T13:03:21Z

composables/project_manager.js

+          .then(function () {
+            return hybridViewerStore.importStores(snapshot.hybridViewer)
+          })
+          .then(function () {


_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

github-actions · 2025-11-12T13:03:21Z

composables/project_manager.js

+          .then(function () {
+            return hybridViewerStore.initHybridViewer()
+          })
+          .then(function () {


_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

github-actions · 2025-11-12T13:03:21Z

composables/project_manager.js

+          .then(function () {
+            return treeviewStore.importStores(snapshot.treeview)
+          })
+          .then(function () {


_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

github-actions · 2025-11-12T13:03:21Z

composables/project_manager.js

+        treeviewStore.isImporting = true
+
+        return Promise.resolve()
+          .then(function () {


_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

github-actions · 2025-11-12T13:03:21Z

composables/project_manager.js

+
+        treeviewStore.isImporting = true
+
+        return Promise.resolve()


_{reported by reviewdog 🐶}
Prefer await to then()/catch()/finally()

github-actions · 2025-11-12T15:21:48Z

utils/default_styles.js

 const lines_defaultColor = { r: 20, g: 20, b: 20 }
 const surfaces_defaultVisibility = true
-const surfaces_defaultColor = { r: 20, g: 20, b: 20 }
+const surfaces_defaultColor = { r: 255, g: 255, b: 255 }


_{reported by reviewdog 🐶}
Identifier name is too short (< 2).

github-actions · 2025-11-12T15:21:48Z

utils/default_styles.js

 const lines_defaultColor = { r: 20, g: 20, b: 20 }
 const surfaces_defaultVisibility = true
-const surfaces_defaultColor = { r: 20, g: 20, b: 20 }
+const surfaces_defaultColor = { r: 255, g: 255, b: 255 }


_{reported by reviewdog 🐶}
Identifier name is too short (< 2).

github-actions · 2025-11-12T15:21:48Z

utils/default_styles.js

 const lines_defaultColor = { r: 20, g: 20, b: 20 }
 const surfaces_defaultVisibility = true
-const surfaces_defaultColor = { r: 20, g: 20, b: 20 }
+const surfaces_defaultColor = { r: 255, g: 255, b: 255 }


_{reported by reviewdog 🐶}
Identifier name is too short (< 2).

github-actions · 2025-11-12T15:21:48Z

stores/data_base.js

+  function exportStores() {
+    const snapshotDb = {}
+    for (const [id, item] of Object.entries(db)) {
+      if (!item) continue


_{reported by reviewdog 🐶}
Expected { after 'if' condition.

github-actions · 2025-11-12T15:21:48Z

stores/data_base.js

    return flat_indexes
  }

+  function exportStores() {


_{reported by reviewdog 🐶}
Expected a function expression.

github-actions · 2025-11-12T15:21:50Z

stores/data_style.js

    }
  }

+  const setModelEdgesVisibility = (id, visibility) => {


_{reported by reviewdog 🐶}
Unexpected block statement surrounding arrow body; move the returned value immediately after the =>.

github-actions · 2025-11-12T15:21:50Z

stores/data_style.js

+    return modelStyleStore.setModelEdgesVisibility(id, visibility)
+  }
+
+  const modelEdgesVisibility = (id) => {


_{reported by reviewdog 🐶}
Unexpected block statement surrounding arrow body; move the returned value immediately after the =>.

github-actions · 2025-11-12T15:21:50Z

stores/data_style.js

+    return modelStyleStore.modelEdgesVisibility(id)
+  }
+
+  const exportStores = () => {


_{reported by reviewdog 🐶}
Unexpected block statement surrounding arrow body; move the returned value immediately after the =>.

github-actions · 2025-11-12T15:21:50Z

stores/data_style.js

  const meshStyleStore = useMeshStyle()
  const modelStyleStore = useModelStyle()
  const dataBaseStore = useDataBaseStore()
+  const hybridViewerStore = useHybridViewerStore()


_{reported by reviewdog 🐶}
Variable 'hybridViewerStore' is declared but never used. Unused variables should start with a '_'.

github-actions · 2025-11-12T15:21:50Z

stores/data_style.js

+    const promises = []
+    for (const id of ids) {
+      const meta = dataBaseStore.itemMetaDatas(id)
+      const objectType = meta?.object_type


_{reported by reviewdog 🐶}
Optional chaining is not allowed.

github-actions · 2025-11-12T16:19:44Z

utils/file_import_workflow.js

 }

-export { importFile, importWorkflow }
+async function importItemFromSnapshot(item) {


_{reported by reviewdog 🐶}
Expected a function expression.

github-actions · 2025-11-12T16:19:44Z

utils/file_import_workflow.js

+  return item.id
+}
+
+async function importWorkflowFromSnapshot(items) {


_{reported by reviewdog 🐶}
Expected a function expression.

github-actions · 2025-11-12T16:19:44Z

utils/file_import_workflow.js

+
+  const ids = []
+  for (const item of items) {
+    const id = await importItemFromSnapshot(


_{reported by reviewdog 🐶}
Unexpected await inside a loop.

github-actions · 2025-11-12T16:19:44Z

utils/file_import_workflow.js

+}
+
+async function importWorkflowFromSnapshot(items) {
+  console.log("[importWorkflowFromSnapshot] start", { count: items?.length })


_{reported by reviewdog 🐶}
Unexpected console statement.

github-actions · 2025-11-12T16:19:44Z

utils/file_import_workflow.js

+    ids.push(id)
+  }
+  hybridViewerStore.remoteRender()
+  console.log("[importWorkflowFromSnapshot] done", { ids })


_{reported by reviewdog 🐶}
Unexpected console statement.

github-actions · 2025-11-12T16:19:44Z

utils/file_import_workflow.js

+  return ids
+}
+
+export { importFile, importWorkflow, importWorkflowFromSnapshot }


_{reported by reviewdog 🐶}
Named exports are not allowed.

github-actions · 2025-11-12T16:19:44Z

utils/file_import_workflow.js

 }

-export { importFile, importWorkflow }
+async function importItemFromSnapshot(item) {


_{reported by reviewdog 🐶}
async is not allowed

github-actions · 2025-11-12T16:19:45Z

utils/file_import_workflow.js

+  return item.id
+}
+
+async function importWorkflowFromSnapshot(items) {


_{reported by reviewdog 🐶}
async is not allowed

github-actions · 2025-11-12T16:19:45Z

utils/file_import_workflow.js

+}
+
+async function importWorkflowFromSnapshot(items) {
+  console.log("[importWorkflowFromSnapshot] start", { count: items?.length })


_{reported by reviewdog 🐶}
Optional chaining is not allowed.

feat(save_and_load): add save/load methods to all stores for project …

aa86c91

…persistence create project manager plugin for import/export operations update tests for new functionality and async loading

MaxNumerique requested a review from JulienChampagnol October 28, 2025 08:10

MaxNumerique and others added 6 commits October 28, 2025 08:11

Apply prepare changes

5b2b4f3

cartStore becomes geodeStore

f4ef491

Merge branch 'feat/save_and_load' of https://github.com/Geode-solutio…

5f3e0c6

…ns/OpenGeodeWeb-Front into feat/save_and_load

Apply prepare changes

e6c54b5

renames and move tests

01a9732

Merge branch 'next' of https://github.com/Geode-solutions/OpenGeodeWe…

0b70454

…b-Front into feat/save_and_load

BotellaA reviewed Oct 28, 2025

View reviewed changes

plugins/project_manager.js Outdated Show resolved Hide resolved

MaxNumerique and others added 8 commits October 29, 2025 10:31

persist upload folder

d577721

renamed app_store store to app

df419ec

rm unused var

bb9d02d

importStore & exportStore refacto

a93225c

Apply prepare changes

af41f9d

testes with import/export

ea63e2b

Merge branch 'feat/save_and_load' of https://github.com/Geode-solutio…

97072b5

…ns/OpenGeodeWeb-Front into feat/save_and_load

Apply prepare changes

63464b7

MaxNumerique changed the title ~~feat(save_and_load): add save/load methods to all stores for project …~~ feat(save_and_load): add import/export methods to all stores for project … Oct 30, 2025

MaxNumerique changed the title ~~feat(save_and_load): add import/export methods to all stores for project …~~ feat(save_and_load): add import/export methods Oct 30, 2025

MaxNumerique requested a review from BotellaA October 30, 2025 15:40

MaxNumerique and others added 6 commits October 30, 2025 17:02

test

98991cf

Merge branch 'feat/save_and_load' of https://github.com/Geode-solutio…

6321c8f

…ns/OpenGeodeWeb-Front into feat/save_and_load

Apply prepare changes

35c75dd

test

22a67fe

Merge branch 'feat/save_and_load' of https://github.com/Geode-solutio…

caf5a39

…ns/OpenGeodeWeb-Front into feat/save_and_load

Apply prepare changes

18fc885

BotellaA reviewed Oct 30, 2025

View reviewed changes

JulienChampagnol approved these changes Oct 31, 2025

View reviewed changes

composables/project_manager.js Outdated Show resolved Hide resolved

composables/project_manager.js Outdated Show resolved Hide resolved

tests/unit/plugins/ProjectLoad.nuxt.test.js Outdated Show resolved Hide resolved

stores/data_base.js Outdated Show resolved Hide resolved

MaxNumerique and others added 2 commits October 31, 2025 11:46

exportStores/importStores and createTestingPinia

d4a1cd6

Apply prepare changes

8aec8b8

Apply prepare changes

1ad0cae

github-actions bot reviewed Nov 11, 2025

View reviewed changes

MaxNumerique added 2 commits November 11, 2025 20:30

refacto import

a9222e8

Merge branch 'feat/save_and_load' of https://github.com/Geode-solutio…

c58d78d

…ns/OpenGeodeWeb-Front into feat/save_and_load

github-actions bot reviewed Nov 11, 2025

View reviewed changes

color

f20ac1a

github-actions bot reviewed Nov 11, 2025

View reviewed changes

removed com

16a5aa9

github-actions bot reviewed Nov 11, 2025

View reviewed changes

project_load test

10dcaad

github-actions bot reviewed Nov 11, 2025

View reviewed changes

test treeview

c2859b1

github-actions bot reviewed Nov 12, 2025

View reviewed changes

MaxNumerique added 3 commits November 12, 2025 09:55

test hybrid_vieer

dde3190

hybrid_viewer, default-style and data_style lint

b213a23

exportProject no longer triggers multiple times

edc66de

github-actions bot reviewed Nov 12, 2025

View reviewed changes

MaxNumerique and others added 3 commits November 12, 2025 12:24

grid scale & zscaling fix

2c53c79

Merge branch 'next' of https://github.com/Geode-solutions/OpenGeodeWe…

32611bb

…b-Front into feat/save_and_load

Apply prepare changes

3d89884

github-actions bot reviewed Nov 12, 2025

View reviewed changes

MaxNumerique added 2 commits November 12, 2025 14:01

revert

4978f96

Merge branch 'feat/save_and_load' of https://github.com/Geode-solutio…

60ef7d1

…ns/OpenGeodeWeb-Front into feat/save_and_load

github-actions bot reviewed Nov 12, 2025

View reviewed changes

MaxNumerique added 2 commits November 12, 2025 16:19

fixes remoteRender

fb2818a

fixes remoteRender

b53db94

github-actions bot reviewed Nov 12, 2025

View reviewed changes

MaxNumerique and others added 2 commits November 12, 2025 17:17

revert

4a88bda

Apply prepare changes

c0419f9

github-actions bot reviewed Nov 12, 2025

View reviewed changes


		const { viewer_call } = await import("@/composables/viewer_call.js")

		expect(infraStoreMock.create_connection).toHaveBeenCalled()

feat(save_and_load): add import/export methods #249

Are you sure you want to change the base?

feat(save_and_load): add import/export methods #249

Uh oh!

Conversation

MaxNumerique commented Oct 28, 2025

Uh oh!

Uh oh!

BotellaA Oct 30, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions bot left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Nov 11, 2025

Choose a reason for hiding this comment

Uh oh!